Skip to content

Conversation

@cpsievert
Copy link
Contributor

@cpsievert cpsievert commented Nov 14, 2025

Closes #69

Base automatically changed from feat/py-new-api to main November 18, 2025 22:27
@cpsievert cpsievert force-pushed the feat/py-app-method branch 5 times, most recently from 60e9b32 to 8e699a5 Compare November 18, 2025 22:47
@cpsievert cpsievert requested a review from Copilot November 18, 2025 22:53
@cpsievert cpsievert marked this pull request as ready for review November 18, 2025 22:54

This comment was marked as resolved.

@cpsievert cpsievert requested a review from gadenbuie November 18, 2025 22:57
@cpsievert cpsievert requested a review from gadenbuie November 20, 2025 18:32
Copy link
Contributor

@gadenbuie gadenbuie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you!

# Reactive values to store state
sql = ReactiveString("")
title = ReactiveStringOrNone(None)
has_greeted = reactive.value[bool](False) # noqa: FBT003
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't remember this from the R side, are we missing it over there?

Copy link
Contributor Author

@cpsievert cpsievert Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe so, yea. Filed #107 to make sure we don't forget

@cpsievert cpsievert merged commit d3479cb into main Nov 24, 2025
6 checks passed
@cpsievert cpsievert deleted the feat/py-app-method branch November 24, 2025 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Python] Add querychat.app() function

3 participants